Skip to content

chore: update spam list#2226

Merged
manishdait merged 2 commits into
hiero-ledger:mainfrom
bhuvan-somisetty:fix/2222-update-spam-list
May 4, 2026
Merged

chore: update spam list#2226
manishdait merged 2 commits into
hiero-ledger:mainfrom
bhuvan-somisetty:fix/2222-update-spam-list

Conversation

@bhuvan-somisetty

@bhuvan-somisetty bhuvan-somisetty commented May 3, 2026 •
edited by exploreriii
Loading

Copy link
Copy Markdown
Contributor

Description:

Related issue(s):

Fixes #2222

Notes for reviewer:

Checklist

  • Documented (Code comments, README, etc.)
  • Tested (unit, integration, etc.)

@bhuvan-somisetty
bhuvan-somisetty requested review from a team as code owners May 3, 2026 15:46
@coderabbitai

coderabbitai Bot commented May 3, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Walkthrough

A spam list entry at .github/spam-list.txt is corrected, replacing fauds-dev101 with Fouad-dev101 on line 14.

Changes

Spam List Correction

Layer / File(s) Summary
Configuration Update
.github/spam-list.txt		 Entry on line 14 updated from fauds-dev101 to Fouad-dev101.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive The PR description is incomplete with only placeholder text and no actual summary of changes provided by the author. Provide a clear one or two-line summary describing what the PR does and why it's needed, filling in the placeholder sections of the description template.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title 'chore: update spam list' is directly related to the changeset which updates an entry in the .github/spam-list.txt file.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

  • Generate code and open pull requests
  • Plan features and break down work
  • Investigate incidents and troubleshoot customer tickets together
  • Automate recurring tasks and respond to alerts with triggers
  • Summarize progress and report instantly

Built for teams:

  • Shared memory across your entire org—no repeating context
  • Per-thread sandboxes to safely plan and execute work
  • Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@codacy-production

codacy-production Bot commented May 3, 2026 •
edited
Loading

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

exploreriii
exploreriii requested changes May 3, 2026
View reviewed changes
Comment thread .github/spam-list.txt Outdated
@exploreriii
exploreriii marked this pull request as draft May 3, 2026 16:34
@bhuvan-somisetty

bhuvan-somisetty commented May 3, 2026

Copy link
Copy Markdown
Contributor Author

Hi @exploreriii, removed the duplicate misspelled entry as requested. Please take another look when you get a chance!

@bhuvan-somisetty
bhuvan-somisetty force-pushed the fix/2222-update-spam-list branch from 729d3d8 to 621789c Compare May 3, 2026 18:47
@exploreriii exploreriii added reviewer: community pull requests looking for community reviews step: 1st 1st stage of the review approval process labels May 4, 2026
@exploreriii
exploreriii marked this pull request as ready for review May 4, 2026 12:31
Copilot AI review requested due to automatic review settings May 4, 2026 12:31
Copilot AI reviewed May 4, 2026
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the repository’s GitHub spam list to correct/refresh an entry, helping automated repo hygiene workflows continue to filter known spam accounts accurately.

Changes:

  • Replace the existing fauds-dev101 entry with Fouad-dev101 in the spam list.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

NssGourav
NssGourav approved these changes May 4, 2026
View reviewed changes

@NssGourav NssGourav left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, thanks for addressing the requested update.
Please rebase with the latest main before merge.

bhuvan-somisetty and others added 2 commits May 4, 2026 20:21
@bhuvan-somisetty
chore: update spam list
44d81a0 
Signed-off-by: Bhuvan Somisetty <somisettybhuvan5@gmail.com>
@bhuvan-somisetty @claude
fix: remove duplicate misspelled entry fauds-dev101 from spam list
02b21fe 
Signed-off-by: bhuvan-somisetty <somisettybhuvan5@gmail.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@bhuvan-somisetty
bhuvan-somisetty force-pushed the fix/2222-update-spam-list branch from 621789c to 02b21fe Compare May 4, 2026 14:52
@exploreriii exploreriii added step: 2nd second stage of the review approval process and removed step: 1st 1st stage of the review approval process labels May 4, 2026
@manishdait manishdait added this to the v0.2.7 milestone May 4, 2026
@manishdait
manishdait merged commit cf653c3 into hiero-ledger:main May 4, 2026
15 checks passed
@github-actions github-actions Bot added notes: automated notes: spam-list-update involves updating the spam list skill: beginner Achievable by a fairly new comer that has already completed a couple of good first issues labels May 4, 2026
NssGourav pushed a commit to NssGourav/hiero-sdk-python that referenced this pull request May 14, 2026
@bhuvan-somisetty @claude @NssGourav
chore: update spam list (hiero-ledger#2226)
b3345cb 
Signed-off-by: Bhuvan Somisetty <somisettybhuvan5@gmail.com>
Signed-off-by: bhuvan-somisetty <somisettybhuvan5@gmail.com>
Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments
@manishdait manishdait manishdait approved these changes
@exploreriii exploreriii exploreriii approved these changes
@nadineloepfe nadineloepfe Awaiting requested review from nadineloepfe nadineloepfe is a code owner automatically assigned from hiero-ledger/hiero-sdk-python-maintainers
@PavelSBorisov PavelSBorisov Awaiting requested review from PavelSBorisov PavelSBorisov is a code owner automatically assigned from hiero-ledger/github-maintainers
+1 more reviewer
@NssGourav NssGourav NssGourav approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

notes: automated notes: spam-list-update involves updating the spam list reviewer: community pull requests looking for community reviews skill: beginner Achievable by a fairly new comer that has already completed a couple of good first issues step: 2nd second stage of the review approval process

Projects

None yet

Milestone

v0.2.7

Development

Successfully merging this pull request may close these issues.

Update spam list (1 additions, 0 removals)

5 participants

@bhuvan-somisetty @manishdait @exploreriii @NssGourav